Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Vrf] Disable swss warm_boot flag after test_vrf_swss_warm_reboot #4636

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

AndriiLozovyi
Copy link
Contributor

Signed-off-by: Andrii-Yosafat Lozovyi andrii-yosafatx.lozovyi@intel.com

Description of PR

Summary: After PR - 1460 additional check was added to ensure, that before issuing an warm_reboot there are no warm_reboot flags set. test_vrf_swss_warm_reboot enables warm_boot flag for swss by using cmd - config warm_restart enable swss, but after restart of swss, next test case fails to make warm_reboot of system due to this flag is still set.

Error message:
Warm restart flag for swss is set. Please check if a warm restart for swss is in progress.

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

Disable swss warm_boot flag after test_vrf_swss_warm_reboot.

How did you do it?

How did you verify/test it?

Run vrf/test_vrf.py::TestVrfWarmReboot, test_vrf_system_warm_reboot gets executed.

Any platform specific information?

SONiC Software Version: SONiC.master.48366-dirty-20211103.084953
Distribution: Debian 10.11
Kernel: 4.19.0-12-2-amd64
Build commit: 689c10109
Build date: Wed Nov  3 15:37:52 UTC 2021
Built by: AzDevOps@sonic-build-workers-000UT4
Platform: x86_64-arista_7170_64c
HwSKU: Arista-7170-64C

Supported testbed topology if it's a new test case?

Documentation

Signed-off-by: Andrii-Yosafat Lozovyi <andrii-yosafatx.lozovyi@intel.com>
@AndriiLozovyi AndriiLozovyi requested a review from a team as a code owner November 5, 2021 10:29
@wangxin
Copy link
Collaborator

wangxin commented Nov 12, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit 408b2c8 into sonic-net:master Nov 17, 2021
AntonHryshchuk pushed a commit to AntonHryshchuk/sonic-mgmt that referenced this pull request Jan 4, 2022
…nic-net#4636)

What is the motivation for this PR?
After PR - sonic-net/sonic-utilities#1460 additional check was added to ensure, that before issuing an warm_reboot there are no warm_reboot flags set. test_vrf_swss_warm_reboot enables warm_boot flag for swss by using cmd - config warm_restart enable swss, but after restart of swss, next test case fails to make warm_reboot of system due to this flag is still set. Error message:
Warm restart flag for swss is set. Please check if a warm restart for swss is in progress.

How did you do it?
Disable swss warm_boot flag after test_vrf_swss_warm_reboot.

How did you verify/test it?
Run vrf/test_vrf.py::TestVrfWarmReboot, test_vrf_system_warm_reboot gets executed.

Signed-off-by: Andrii-Yosafat Lozovyi <andrii-yosafatx.lozovyi@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants